Skip to content

Export gzfwrite() #6880

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Export gzfwrite() #6880

wants to merge 1 commit into from

Conversation

cmb69
Copy link
Member

@cmb69 cmb69 commented Apr 19, 2021

Xdebug recently introduced the possibility to compress profile
information and this uses gzfwrite(), what is not yet exported on
Windows. That would require run-time linking to be worked around.


Alternatively, we could fully update to https://github.com/winlibs/zlib/blob/master/win32/zlib.def.

As 7.4 RM and Xdebug maintainer, @derickr might be interested in this.

Xdebug recently introduced the possibility to compress profile
information and this uses `gzfwrite()`, what is not yet exported on
Windows.  That would require run-time linking to be worked around.
Copy link
Member

@nikic nikic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't see any reason not to do this...

@cmb69
Copy link
Member Author

cmb69 commented Apr 26, 2021

Well, might not actually be necessary for Xdebug, but maybe for others.

@cmb69 cmb69 closed this in a277129 Apr 26, 2021
@cmb69 cmb69 deleted the cmb/gzfwrite branch April 26, 2021 10:14
cmb69 added a commit to cmb69/xdebug that referenced this pull request Sep 15, 2021
`gzfwrite` is not exported prior to PHP 7.4.20, see
<php/php-src#6880>.
derickr pushed a commit to xdebug/xdebug that referenced this pull request Sep 21, 2021
`gzfwrite` is not exported prior to PHP 7.4.20, see
<php/php-src#6880>.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants